Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set the output of %stdout %stderr and %result from executed tasks #26

Merged
merged 1 commit into from
Jan 5, 2020

Conversation

Archenoth
Copy link
Contributor

This pull request adds the ability to return data from termux-app using PendingIntents.

It uses them to set the executed process %result, %stdout, and %stderr for Tasker to use.

nano, editing a script that will output to stdout, stderr, and exit with a status of 20

The Tasker configuration screen showing that there are three variables that will be set, %stdout, %stderr, and %result

A notification from Tasker showing the stdout, the stderr, and the exit status of 20

(This pull request requires the changes in this pull request in the main app to function properly and solves this issue.)

This commit adds the ability to return data from the termux-app
TermuxService with a PendingIntent response

It also provides a simple implementation of using it to return the
process exit status, stdout and stderr outputs of the executed command
for Tasker to use
@Archenoth
Copy link
Contributor Author

Oh, actually, this also has the side effect of solving this issue! Perhaps not as a checkbox, but commands will no longer run asynchronously unless you set their timeout to "None", in which case they will revert back to continuing immediately.

@Archenoth
Copy link
Contributor Author

For the sake of collecting everything relevant, this also appears to be the subject of this issue, and should fix it too.

@fornwall fornwall merged commit 27b442d into termux:master Jan 5, 2020
@fornwall
Copy link
Member

fornwall commented Jan 5, 2020

Thanks a lot!

@fornwall
Copy link
Member

fornwall commented Jan 5, 2020

This has been merged and released in version 0.4 of the Termux:Tasker app - great job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants